Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: use standard #661

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Conversation

mperham
Copy link

@mperham mperham commented Jan 20, 2025

Integrate the Standard linter into the build.

I've included both safe and unsafe automated fixes but I did not make any manual code changes. Please run bundle exec rake standard to see the list of lint warnings.

I also included a .standard.yml with ruby 2.3.0 as the minimum version. If you bump this, it will use newer language features. Unfortunately my local rubies are mostly horked so I could only test Ruby 3.3.

mperham and others added 7 commits January 20, 2025 10:26
This way it won't get run for every appraisal. We'll run it separately via Lefthook in dev and a separate workflow in CI.
Rakefile:15: warning: rdoc/task was loaded from the standard library, but is not part of the default gems starting from Ruby 3.5.0.
@joshuap
Copy link
Member

joshuap commented Jan 23, 2025

@mperham where were you seeing Ruby 2.3 builds happening? Our test matrix starts at 3.0:

https://github.com/honeybadger-io/honeybadger-ruby/blob/master/.github/workflows/ruby.yml#L17

We can remove 3.0 and (soon) 3.1 though, since they are EOL:

https://www.ruby-lang.org/en/downloads/branches/

cc @stympy

@mperham
Copy link
Author

mperham commented Jan 23, 2025

I was only speaking of the required Ruby version in the gemspec. I saw 2.3 but realized I couldn't go anywhere near that far back to test.

@joshuap
Copy link
Member

joshuap commented Jan 23, 2025

I was only speaking of the required Ruby version in the gemspec. I saw 2.3 but realized I couldn't go anywhere near that far back to test.

Ah, right. @stympy I think we should update the gemspec before merging this so that we can use more modern operators (I think we may have already been using some?).

@stympy stympy changed the title Use standard chore: use standard Jan 23, 2025
@joshuap
Copy link
Member

joshuap commented Feb 13, 2025

@stympy JRuby is a tricky beast, it might make sense to update the matrix in a separate issue 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants